Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[fix] print self in warning. #15614

Merged
merged 3 commits into from
Jul 21, 2019
Merged

[fix] print self in warning. #15614

merged 3 commits into from
Jul 21, 2019

Conversation

kshitij12345
Copy link
Contributor

Before this:

/home/user/Desktop/Repositories/mxnet/incubator-mxnet/python/mxnet/gluon/block.py:845: UserWarning: "{}" is being hybridized while still having forward hook/pre-hook. If "{}" is a child of HybridBlock, the hooks will not take effect.
  warnings.warn('"{}" is being hybridized while still having forward hook/pre-hook. '

After this:

/home/user/Desktop/Repositories/mxnet/incubator-mxnet/python/mxnet/gluon/block.py:847: UserWarning: "Dense(120 -> 84, linear)" is being hybridized while still having forward hook/pre-hook. If "Dense(120 -> 84, linear)" is a child of HybridBlock, the hooks will not take effect.
  .format(self, self))

@kshitij12345 kshitij12345 requested a review from szha as a code owner July 19, 2019 19:28
@karan6181
Copy link
Contributor

@mxnet-label-bot add [Gluon, pr-awaiting-merge]

@marcoabreu marcoabreu added Gluon pr-awaiting-merge Review and CI is complete. Ready to Merge labels Jul 19, 2019
@wkcn wkcn merged commit 2b4c9c0 into apache:master Jul 21, 2019
@wkcn
Copy link
Member

wkcn commented Jul 21, 2019

Merged. Thanks for your contribution!

anirudhacharya pushed a commit to anirudhacharya/mxnet that referenced this pull request Aug 20, 2019
@kshitij12345
[fix] print self in warning. (apache#15614)
0341eb9 
* use format

* make pylint happy

* Update block.py
@kshitij12345 kshitij12345 deleted the fix/print-msg-hybrid-block branch September 7, 2019 09:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@marcoabreu marcoabreu marcoabreu approved these changes

@szha szha szha approved these changes

Assignees
No one assigned
Labels
Gluon pr-awaiting-merge Review and CI is complete. Ready to Merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kshitij12345 @karan6181 @wkcn @szha @marcoabreu